Skip to content

[opentelemetry-collector]: custom apiVersion support#2050

Merged
TylerHelmuth merged 7 commits into
open-telemetry:mainfrom
medzin:main
Apr 10, 2026
Merged

[opentelemetry-collector]: custom apiVersion support#2050
TylerHelmuth merged 7 commits into
open-telemetry:mainfrom
medzin:main

Conversation

@medzin
Copy link
Copy Markdown
Contributor

@medzin medzin commented Jan 26, 2026

Adds configurable apiVersion and extra updateStrategy fields for DaemonSet/StatefulSet/Deployment mode. This enables:

  • Using custom controllers like OpenKruise Advanced DaemonSet and Advanced StatefulSet
  • Partition-based rollouts for safer canary deployments
  • In-place pod updates without recreation
  • Testing new API versions before they become stable

@medzin medzin requested review from a team, TylerHelmuth, dmitryax and povilasv as code owners January 26, 2026 10:44
Comment thread charts/opentelemetry-collector/values.yaml Outdated
@medzin medzin changed the title [opentelemetry-collector]: custom DaemonSet support [opentelemetry-collector]: custom apiVersion support Mar 21, 2026
@medzin medzin requested a review from TylerHelmuth March 31, 2026 13:18
Copy link
Copy Markdown
Member

@TylerHelmuth TylerHelmuth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left some feedback on how we can make the implementation simpler. Also please remove the new examples as the new field is straightforward.

Comment thread charts/opentelemetry-collector/values.yaml Outdated
Comment thread charts/opentelemetry-collector/values.yaml Outdated
Comment thread charts/opentelemetry-collector/values.schema.json
Comment thread charts/opentelemetry-collector/values.schema.json Outdated
@medzin medzin force-pushed the main branch 4 times, most recently from c7595a2 to 5c52829 Compare April 9, 2026 17:12
@medzin
Copy link
Copy Markdown
Contributor Author

medzin commented Apr 9, 2026

@TylerHelmuth removed examples and simplified the code.

medzin added 6 commits April 10, 2026 21:09
Adds configurable apiVersion and extra updateStrategy fields for
DaemonSet mode. This enables:

- Using custom controllers like OpenKruise Advanced DaemonSet
- Partition-based rollouts for safer canary deployments
- In-place pod updates without recreation
- Testing new API versions before they become stable

Includes example and CI values for OpenKruise configuration.
Move .Values.daemonset.apiVersion to .Values.apiVersion so it applies
uniformly to daemonset, deployment, and statefulset modes.
@TylerHelmuth TylerHelmuth merged commit 3bb35fb into open-telemetry:main Apr 10, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants